Skip to content

[pull] master from supabase:master#1

Merged
pull[bot] merged 10 commits intocode:masterfrom
supabase:master
Apr 25, 2024
Merged

[pull] master from supabase:master#1
pull[bot] merged 10 commits intocode:masterfrom
supabase:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Apr 25, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

saltcod and others added 10 commits April 24, 2024 21:00
* Update CTA for manual disk size increase beyond limit

* Remove unused imports

* Add error handler for sql editor

* Fix clicking supabase icon button in project not closing the nav bar when entering into another project from the home page

* Update copy
#23235)

* enhance(sql editor): set result column width roughly based on contents

* Small fix

---------

Co-authored-by: Joshen Lim <joshenlimek@gmail.com>
remove new line in the middle of a sentence
Exposes the API routes for Realtime appropriately via Kong configuration.

Also alters the analytics configuration to avoid extra logs due to having only one analytics server instead of a cluster

Co-authored-by: Rodrigo Mansueli <rodrigo@mansueli.com>
* chore: update copy on signup and reset password links

* fix: add config for updating otp length
* Rename OPENAI_KEY to OPENAI_API_KEY

* Force a redeploy
)

* Fix the header when editing a function.

* Parse the arguments properly.
@pull pull Bot added the ⤵️ pull label Apr 25, 2024
@pull pull Bot merged commit 09c397f into code:master Apr 25, 2024
pull Bot pushed a commit that referenced this pull request Mar 6, 2026
…irmationDialog (supabase#43430)

## What kind of change does this PR introduce?

Form handling improvement.

## What is the current behavior?

supabase#43201 standardised our
discard changes behaviour with a shared hook and
`DiscardChangesConfirmationDialog` component. But many forms and sheets
still:

1. Don’t have any Discard-confirm close behaviour, making it too easy to
make accidental discards
2. Use a more complicated, manually-created `CloseConfirmationModal`
approach

## What is the new behavior?

- Replaced all instances of `#2` above that had `CloseConfirmationModal`
with `DiscardChangesConfirmationDialog` and its hook
- Improved design system documentation around dirty form dismissal

| Before | After |
| --- | --- |
| <img width="987" height="569" alt="Mercor Apexroles Foo
Supabase-9A40EC7C-F335-4B26-B567-450FC0845463"
src="https://github.com/user-attachments/assets/363bed82-34d2-4cc8-9164-6d18cfdbdbbc"
/> | <img width="987" height="569" alt="Mercor Apexroles Foo
Supabase-F427F1FA-DECC-4194-B663-A9E5A6F285A1"
src="https://github.com/user-attachments/assets/d49fafdc-a5c2-46df-9b67-ec42bacbe716"
/> |

## To test

Try editing values these sheets in staging, then blurring the sheet or
pressing `esc`:

- CreateQueueSheet.tsx
- CronJobsTab.tsx
- CronJobPage.tsx
- EditWrapperSheet.tsx
- OverviewTab.tsx
- WrappersTab.tsx
- CreateFunction/index.tsx
- EditHookPanel.tsx
- TriggerSheet.tsx
- SidePanelEditor.tsx
- EditSecretSheet.tsx
- PolicyEditorModal/index.tsx
- PolicyEditorPanel/index.tsx

## Still to come

- [ ] Incrementally take on `#1`: implement
`DiscardChangesConfirmationDialog` and its hook in sheets or dialog
forms that have no dirty form dismissal handling

---------

Co-authored-by: Joshen Lim <joshenlimek@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants